Skip to content

ci: 로깅 설정 및 zipkin 설정 추가 / #51#52

Merged
JayongLee merged 3 commits intomainfrom
ci/#51
Nov 28, 2025
Merged

ci: 로깅 설정 및 zipkin 설정 추가 / #51#52
JayongLee merged 3 commits intomainfrom
ci/#51

Conversation

@JayongLee
Copy link
Member

@JayongLee JayongLee commented Nov 28, 2025

🗞️ 연관된 이슈

🔥 이슈번호

✅ 작업 내용

  • 로깅 설정 및 zipkin 설정 추가

📸 스크린샷 (선택)

체크리스트 ✅

  • 코드가 정상적으로 컴파일되나요?
  • merge할 브랜치의 위치를 확인했나요?
  • 테스트 코드를 작성하셨나요?

기타

@JayongLee JayongLee self-assigned this Nov 28, 2025
@gemini-code-assist
Copy link

Summary of Changes

Hello @JayongLee, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 애플리케이션의 로깅 시스템을 개선하고 분산 트레이싱 기능을 도입하여 개발 및 운영 환경에서의 가시성을 높이는 것을 목표로 합니다. 기존의 간단한 로깅 설정을 확장하여 logback-dev.xml을 통한 세밀한 제어를 가능하게 하고, Zipkin을 활용한 트레이싱을 통해 요청 흐름을 쉽게 추적할 수 있도록 기반을 마련했습니다.

Highlights

  • 로깅 설정 변경: 기존 application-dev.yml에 직접 정의된 로깅 레벨 설정을 제거하고, logback-dev.xml 파일을 통해 로깅을 관리하도록 변경했습니다.
  • 분산 트레이싱 추가: micrometer-tracing-bridge-brave 및 zipkin-reporter-brave 의존성을 추가하고, application-dev.yml에 Zipkin 트레이싱 설정을 포함하여 분산 트레이싱 기능을 도입했습니다.
  • 커스텀 Logback 설정: logback-dev.xml 파일을 새로 추가하여 개발 환경에 맞는 로깅 패턴(traceId, spanId 포함)과 파일 로깅 정책(롤링 파일, 용량 및 보관 기간 제한)을 정의했습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

이 PR은 로깅 및 분산 추적(Zipkin) 설정을 추가하는 변경 사항을 담고 있습니다. 전반적으로 좋은 방향의 변경이지만, 몇 가지 개선점을 제안합니다. logback-dev.xml 파일에서 로그 파일 경로를 절대 경로 대신 상대 경로로 설정하여 이식성과 일관성을 높이는 것이 중요합니다. 또한, 몇몇 설정 파일 끝에 개행 문자를 추가하여 POSIX 표준을 따르고 잠재적인 파일 처리 문제를 방지하는 것이 좋습니다.

<configuration>
<property name="LOG_PATTERN"
value="%d{yyyy-MM-dd HH:mm:ss.SSS} [%thread] %-5level %logger{36} - [traceId=%X{traceId}, spanId=%X{spanId}] %msg%n"/>
<property name="ACTIVE_LOG" value="/logs/application.log"/>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

로그 파일 경로에 절대 경로('/logs/application.log')를 사용하면 권한 문제나 이식성 문제가 발생할 수 있습니다. 애플리케이션 실행 위치 기준으로 로그 파일이 생성되도록 상대 경로('logs/application.log')를 사용하는 것이 좋습니다.

또한, 현재 설정에서는 활성 로그 파일(<file>)은 절대 경로 /logs에 저장되지만, 롤링된 아카이브 파일(<fileNamePattern>)은 상대 경로 logs에 저장되어 경로가 일치하지 않는 문제가 발생합니다. 상대 경로로 통일하여 일관성을 유지하고 예기치 않은 동작을 방지하는 것이 좋습니다.

Suggested change
<property name="ACTIVE_LOG" value="/logs/application.log"/>
<property name="ACTIVE_LOG" value="logs/application.log"/>

probability: 1.0
zipkin:
tracing:
endpoint: ${ZIPKIN_URL}/api/v2/spans No newline at end of file

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

파일의 마지막에 개행 문자를 추가하는 것을 권장합니다. 이는 텍스트 파일에 대한 POSIX 표준이며, Git과 같은 버전 관리 시스템이나 다른 텍스트 처리 도구와의 호환성을 높여줍니다.

      endpoint: ${ZIPKIN_URL}/api/v2/spans

<appender-ref ref="CONSOLE"/>
<appender-ref ref="FILE"/>
</root>
</configuration> No newline at end of file

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

파일 끝에 개행 문자를 추가하는 것이 좋습니다. 이는 POSIX 표준을 따르는 좋은 관행이며, 일부 도구에서 파일을 처리할 때 예기치 않은 문제를 방지할 수 있습니다.

Suggested change
</configuration>
</configuration>

@JayongLee JayongLee linked an issue Nov 28, 2025 that may be closed by this pull request
1 task
@JayongLee JayongLee merged commit bf9b18f into main Nov 28, 2025
1 check passed
@JayongLee JayongLee deleted the ci/#51 branch November 28, 2025 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: 로깅 설정 및 zipkin 설정 추가

1 participant